Skip to content

Remove test_data usage#10029

Merged
SPPearce merged 15 commits intonf-core:masterfrom
LouisLeNezet:test_data
Feb 16, 2026
Merged

Remove test_data usage#10029
SPPearce merged 15 commits intonf-core:masterfrom
LouisLeNezet:test_data

Conversation

@LouisLeNezet
Copy link
Contributor

PR checklist

Pre requesite of #10016

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the module conventions in the contribution docs
  • If necessary, include test data in your PR.
  • Remove all TODO statements.
  • Broadcast software version numbers to topic: versions - See version_topics
  • Follow the naming conventions.
  • Follow the parameters requirements.
  • Follow the input/output options guidelines.
  • Add a resource label
  • Use BioConda and BioContainers if possible to fulfil software requirements.
  • Ensure that the test works with either Docker / Singularity. Conda CI tests can be quite flaky:
    • For modules:
      • nf-core modules test <MODULE> --profile docker
      • nf-core modules test <MODULE> --profile singularity
      • nf-core modules test <MODULE> --profile conda
    • For subworkflows:
      • nf-core subworkflows test <SUBWORKFLOW> --profile docker
      • nf-core subworkflows test <SUBWORKFLOW> --profile singularity
      • nf-core subworkflows test <SUBWORKFLOW> --profile conda

@LouisLeNezet
Copy link
Contributor Author

Hi Maxime, I had to change a few modules.
Graphtyper is not working well, it produce an empty vcf (the region is too small I guess).
Could you check if it's good for you ?

@LouisLeNezet LouisLeNezet self-assigned this Feb 16, 2026
@SPPearce SPPearce enabled auto-merge February 16, 2026 15:01
@SPPearce SPPearce added this pull request to the merge queue Feb 16, 2026
Merged via the queue into nf-core:master with commit 5e748ff Feb 16, 2026
66 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants